Skip to content

Conversation

@SeddikBellamine
Copy link
Contributor

Description

This PR fixes the tag extraction issue in the dapp-release workflow that was causing Docker build failures.

Problem

The workflow was failing with invalid Docker tag errors because the image-tag variable was empty due to failed tag extraction from git tags.

Solution

Changed the tag extraction method from parsing git tags to reading the version directly from dapp/package.json using grep and sed commands.

Benefits

  • More reliable: Uses the actual version from package.json
  • Simpler: Uses standard shell commands instead of complex tag parsing
  • Consistent: The version in package.json is the source of truth

Files Changed

  • .github/workflows/dapp-release.yml: Updated tag extraction logic

* chore(main): release dapp 0.1.0-alpha.1

* chore: clean up dapp changelog entries

---------

Co-authored-by: iexec-release-please-app[bot] <202620906+iexec-release-please-app[bot]@users.noreply.github.com>
Co-authored-by: SeddikBellamine <[email protected]>
Comment on lines +10 to +11
- rollback debug logging changes from develop branch ([404ce33](https://github.com/iExecBlockchainComputing/web3telegram-sdk/commit/404ce33e7d0705c274155813951284cd1830a3d2))

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- rollback debug logging changes from develop branch ([404ce33](https://github.com/iExecBlockchainComputing/web3telegram-sdk/commit/404ce33e7d0705c274155813951284cd1830a3d2))

this change doesn't seem particularly valuable to include

@PierreJeanjacquot
Copy link
Member

Changes made obsolete by #60

@PierreJeanjacquot PierreJeanjacquot deleted the fix/dapp-release-tag-extraction branch July 29, 2025 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants